Skip to content

fix(web_core): restore listStyle to v0.9 basic catalog ListApi schema - #3956

Merged
gspencergoog merged 2 commits into
a2ui-project:mainfrom
gspencergoog:fix/v0-9-list-style-schema
Nov 5, 2026
Merged

gspencergoog merged 2 commits into
a2ui-project:mainfrom
gspencergoog:fix/v0-9-list-style-schema

Conversation

@gspencergoog

Copy link
Copy Markdown
Collaborator

Summary

Restores the listStyle property ('ordered' | 'unordered' | 'none') to the List component in the A2UI v0.9 basic catalog specification (specification/v0_9/catalogs/basic/catalog.json) and regenerates the v0.9 basic catalog APIs (typescript/core/web_core/src/v0_9/basic_catalog/components/basic_components.ts).

Details

  1. Specification: Added listStyle (string, enum: ['ordered', 'unordered', 'none'], description: "The style of the list (ordered, unordered, or none).") to List in specification/v0_9/catalogs/basic/catalog.json.
  2. Web Core: Regenerated basic_components.ts via generate-catalog.mjs, adding listStyle as an optional enum on ListApi.schema.
  3. Conformance: Updated conformance/test_data/validation_schema/test_v09_basic_validation_schema.json to include listStyle in the golden validation schema for List.
  4. Tests: Added unit tests in List.test.ts verifying that ListApi.schema.parse validates listStyle ('none', 'ordered', 'unordered'), rejects invalid values, and validates list components using PayloadValidator.
  5. Changelog: Documented the change under ## Unreleased in typescript/core/web_core/CHANGELOG.md.

Testing

  • node dist/src/universal/basic_catalog/components/List.test.js (8/8 passing)
  • node dist/src/v0_9/test/verify-schema.test.js (3/3 passing)
  • node tests/conformance/conformance_test.mjs (726/726 passing, 0 failures)
  • corepack yarn workspace @a2ui/web_core build

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request restores the 'listStyle' property to the v0.9 basic catalog 'ListApi' schema. The property is defined as an optional enum with values 'ordered', 'unordered', and 'none'. The changes update the JSON schema, the TypeScript schema definition, and the validation tests to ensure correct parsing and validation of the property. I have no feedback to provide.

@gspencergoog
gspencergoog force-pushed the fix/v0-9-list-style-schema branch from 1902037 to 57a3803 Compare November 3, 2026 21:33
@gspencergoog
gspencergoog force-pushed the fix/v0-9-list-style-schema branch from 57a3803 to f6c414b Compare November 3, 2026 21:39
Resolve CHANGELOG conflicts in dart/a2ui_core, python/a2ui_core, and
typescript/core/web_core by keeping both the listStyle entry and the
entries added on main. The Dart entry stays under Unreleased, above the
new 0.5.0 section.
@gspencergoog
gspencergoog merged commit 2e8a58a into a2ui-project:main Nov 5, 2026
56 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in A2UI Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants